Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

RT updates: remove compiler from labels #129

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

RT updates: remove compiler from labels #129

wants to merge 4 commits into from

Conversation

mkavulich
Copy link
Collaborator

PR Author Checklist:

  • I have linked PR's from all sub-components involved in section below.
  • I am confirming reviews are completed in ALL sub-component PR's.
  • I have run the full RT suite on either Hera/Cheyenne AND have attached the log to this PR below this line:
    • LOG:
  • I have added the list of all failed regression tests to "Anticipated changes" section.
  • I have filled out all sections of the template.

Description

Updating Regression tests so that we no longer have compiler-specific labels, as in the main repository (now all compilers are run by default). Will be adding more to this PR as I fix various issues, needs more testing first.

Linked Issues and Pull Requests

Associated UFSWM Issue to close

None

Subcomponent Pull Requests

None

Blocking Dependencies

None

Subcomponents involved:

  • AQM
  • CDEPS
  • CICE
  • CMEPS
  • CMakeModules
  • FV3
  • GOCART
  • HYCOM
  • MOM6
  • NOAHMP
  • WW3
  • stochastic_physics
  • none

Anticipated Changes

Input data

  • No changes are expected to input data.

Regression Tests:

  • No changes are expected to any regression test.
  • Changes are expected to the following tests:
Tests effected by changes in this PR:

Libraries

  • Not Needed
Code Managers Log
  • This PR is up-to-date with the top of all sub-component repositories except for those sub-components which are the subject of this PR.
  • Move new/updated input data on RDHPCS Hera and propagate input data changes to all supported systems.
    • N/A

Testing Log:

  • RDHPCS
    • Hera
    • Orion
    • Hercules
    • Jet
    • Gaea
    • Cheyenne
  • WCOSS2
    • Dogwood/Cactus
    • Acorn
  • CI
    • Completed
  • opnReqTest
    • N/A
    • Log attached to comment

@mkavulich mkavulich added hera-RT Run regression test on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-RT Run regression test on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 7, 2024
@mkavulich
Copy link
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: RT
[RT] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240207215510/ufs-weather-model
[RT] Log file shows failures.
[RT] Please obtain logs from /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240207215510/ufs-weather-model

@mkavulich mkavulich added hera-RT Run regression test on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-RT Run regression test on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 8, 2024
@mkavulich
Copy link
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: RT
[RT] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240208154010/ufs-weather-model
[RT] Log file shows failures.
[RT] Please obtain logs from /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240208154010/ufs-weather-model

@mkavulich mkavulich added hera-RT Run regression test on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-RT Run regression test on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 8, 2024
@mkavulich
Remove call to delete_pr_dirs() which is messing up test compilation …
1ab2a72 
…somehow, finally clean up the unparseable wall of text logs into something more readable
@mkavulich mkavulich added hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 8, 2024
@mkavulich
Copy link
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: BL
[BL] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240208192114/ufs-weather-model
Please make changes and add the following label back: hera-BL

@mkavulich mkavulich added hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 8, 2024
@mkavulich
Copy link
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: BL
[BL] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240208202113/ufs-weather-model
Please make changes and add the following label back: hera-BL

@mkavulich mkavulich added hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 8, 2024
@mkavulich
Copy link
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: BL
[BL] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240208205712/ufs-weather-model
[BL] ERROR: Baseline location exists before creation:
/scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/new/
Please make changes and add the following label back: hera-BL

@mkavulich mkavulich added hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 8, 2024
@mkavulich
Copy link
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: BL
Please make changes and add the following label back: hera-BL

@mkavulich mkavulich added the hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET label Feb 8, 2024
@mkavulich
Copy link
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: BL
Please make changes and add the following label back: hera-BL

@mkavulich mkavulich added hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 8, 2024
@mkavulich
Copy link
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: BL
Please make changes and add the following label back: hera-BL

@mkavulich mkavulich added the hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET label Feb 8, 2024
@mkavulich
Copy link
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: BL
[BL] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240208213710/ufs-weather-model
Please make changes and add the following label back: hera-BL

@mkavulich mkavulich added hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 8, 2024
@mkavulich
Copy link
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: BL
[BL] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240209001213/ufs-weather-model
[BL] Baseline creation successful
Please make changes and add the following label back: hera-BL

@mkavulich mkavulich added hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 9, 2024
@mkavulich
Copy link
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: BL
[BL] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240209020916/ufs-weather-model
[BL] Baseline creation successful
[RT] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240209023026/ufs-weather-model
Automated RT Failure Notification
Job: BL
[BL] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240209020916/ufs-weather-model
[BL] Baseline creation successful
[RT] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240209023026/ufs-weather-model
[RT] {job_obj.clargs.machine} Job Completed

@mkavulich mkavulich added hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 9, 2024
@mkavulich
Copy link
Collaborator Author

Machine: hera
Job: BL
[BL] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240209030912/ufs-weather-model
[BL] Baseline creation successful
[RT] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240209034711/ufs-weather-model
[RT] hera Job Completed

@mkavulich mkavulich added hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 9, 2024
@mkavulich
Copy link
Collaborator Author

Machine: hera
Job: BL
[BL] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240209184012/ufs-weather-model
Baseline creation successful on hera
[RT] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240209185950/ufs-weather-model
***Regression test successful on hera!

@grantfirl
Copy link
Collaborator

@mkavulich Did you want to commit this next?

@mkavulich
Copy link
Collaborator Author

@grantfirl I will need to bring it up-to-date with the latest code, but if it's okay with you I think that would be a good idea to get this one in next. Once #120 is merged I can update this branch and run a final set of tests/open the PR for review.

@mkavulich
Copy link
Collaborator Author

Okay since tests are running for #121, this will wait until after that one is merged.

@mkavulich mkavulich added hera-RT Run regression test on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-RT Run regression test on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 23, 2024
@mkavulich
Copy link
Collaborator Author

Machine: hera
Job: RT
[RT] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240223233521/ufs-weather-model
Regression test successful on hera!

@grantfirl
Copy link
Collaborator

@mkavulich Can we revive this and get it merged in preparation for running RTs, creating new baselines and catching up with ufs/dev?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mkavulich @grantfirl